Skip to content

Fix YAML indentation #9574

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

Fix YAML indentation #9574

wants to merge 1 commit into from

Conversation

enekochan
Copy link
Contributor

As above.

I've targeted master as the service class definition changed since 2.7.

As above.

I've targeted master as the service class definition changed since 2.7.
@xabbuh
Copy link
Member

xabbuh commented Apr 10, 2018

Thank you @enekochan.

xabbuh added a commit that referenced this pull request Apr 10, 2018
This PR was submitted for the master branch but it was merged into the 3.4 branch instead (closes #9574).

Discussion
----------

Fix YAML indentation

As above.

I've targeted master as the service class definition changed since 2.7.

<!--

If your pull request fixes a BUG, use the oldest maintained branch that contains
the bug (see https://symfony.com/roadmap for the list of maintained branches).

If your pull request documents a NEW FEATURE, use the same Symfony branch where
the feature was introduced (and `master` for features of unreleased versions).

-->

Commits
-------

acbbd85 Fix YAML identation
@xabbuh xabbuh closed this Apr 10, 2018
@xabbuh
Copy link
Member

xabbuh commented Apr 10, 2018

I have merged your changes into the 3.4 branch where I could apply them too. From there we will merged them up.

xabbuh added a commit that referenced this pull request Apr 10, 2018
This PR was merged into the 2.7 branch.

Discussion
----------

fix YAML code block indentation

same as #9574 but on the `2.7` branch

Commits
-------

e55ddf6 fix YAML code block indentation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants